Skip to content

Conversation

@ochafik
Copy link
Contributor

@ochafik ochafik commented Dec 30, 2024

This is to support an exploration of user-configurable message normalization using a template

@ochafik ochafik added the do not merge This PR should not be merged label Jan 14, 2025
@ochafik ochafik mentioned this pull request Jan 19, 2025
@ochafik ochafik marked this pull request as ready for review January 21, 2025 01:05
@ochafik ochafik removed the do not merge This PR should not be merged label Jan 21, 2025
@ochafik ochafik merged commit dd620a9 into main Jan 21, 2025
7 checks passed
@ochafik ochafik deleted the pop branch January 21, 2025 01:17
ochafik referenced this pull request in ochafik/minja Nov 2, 2025
List & object pop() method
CISC pushed a commit to aldehir/minja that referenced this pull request Dec 20, 2025
)

## Summary

Implements support for DeepSeek V3.2's DSML (Domain Specific Markup
Language) format, superseeds google#11 (cc/ @hksdpc255)

DeepSeek V3.2 doesn't provide a Jinja template but uses a custom Python
encoding with DSML format:
```xml
<|DSML|parameter name="key" string="true">value</|DSML|parameter>
```

## Changes

- **Simplified argument needle detection**: Changed from specific
patterns (`"argument_needle":`, `="argument_needle"`) to broader
`"argument_needle"` pattern which matches both JSON keys and DSML
attribute values
- **Local .jinja file support**: Fetch script now handles local `.jinja`
files in MODEL_IDS (for synthetic test templates)
- **Synthetic template**: Added `synthetic-deepseek-v3.2-dsml.jinja`
replicating V3.2's Python encoding logic (from `encoding_dsv32.py`)
- **Integrated testing**: Added synthetic template to MODEL_IDS,
generates 3 test cases (simple, system, tool_use)

## Test plan

- [x] All 248 tests pass
- [x] Capability detection correctly identifies DSML format
(`supports_tool_calls: true`, `requires_object_arguments: true`)
- [x] Synthetic template tests pass for all contexts

Closes google#11

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant